-
Notifications
You must be signed in to change notification settings - Fork 24
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Υπάρχει ένα μικρό regression που μπορεί να αλλάξει εύκολα.
index.html
Outdated
cta_label: "<i class='fa fa-download'></i> Οδηγός Σπουδών 2016-2017" | ||
cta_url: "http://di.ionio.gr/wp-content/uploads/2016/09/OS_2016-2017-final.pdf" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Γιατί υπάρχει αυτή η αλλαγή εδώ που είναι fallback σε πιο παλιό link? Είναι λάθος που ξέφυγε.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Πρεπει να ειναι επειδη χρησιμοποιησα παλιο version του html
Χρησιμοποιήσα παλιότερη version του html. Διόρθωσα το λάθος |
μπράβο, φαίνεται ωραιά @p15zerv @GIANNIS-AGGELIS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Κι εμένα μια χαρά μου φαίνεται. Συμφωνώ ότι το dynamic.html μάλλον είναι καλύτερα να πάει στο theme, αλλά για τώρα μπορεί να γίνει δεκτό κι ας αλλάξει αργότερα.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@constantinexisc @Spirosvw ούτε τοπικά, ούτε στο λινκ του αποθετηρίου δεν βλέπω τις δυναμικές αναρτήσεις??? |
Να εξηγήσω, η δυναμικότητα έιναι βασισμένη στη μεταβλητή site.time, αφού το jekyll είναι static site generator δεν μπορεί να έχει κάποιο άλλο είδος δυναμικότητας ή τυχαιότητας αφότου έχει "χτισει" τις σελίδες και τις κάνει serve. Κάθε φορά που θα γίνεται καινούργιο jekyll serve το site.time θα ενημερώνεται και θα δίνει κάποια άλλα post στην αρχική. |
@constantinexisc η απορία ήταν δικό μου λάθος με κασινγκ της παλιάς σελίδας στον σαφάρι, συγνώμη! |
Συγνώμη για το αργοπορημένο σχόλιο, δεν είχα παρατηρήσει την εξέληξη αυτού του issue... Υπάρχουν μερικά προβλήματα με αυτό το PR κατά την γνώμη μου και του @provopoulos. Πρώτον παρατηρήθηκε οτι πλέον το mobile view στο homepage έχει καταστραφεί καθώς τα κουμπία "Learn more" δεν είναι στις σωστές θέσεις τους. Επιπλέον, υπάρχει άλλο ένα θέμα με την διπλή γραμμή ακριβώς από κάτω από το feature row το οποίο υποψιάζομαι έχέι να κάνει με κάποιο λανθασμένο div nesting. Τέλος με το να υπάρχει το include σε αυτό το αποθετήριο δεν νομίζω οτι αποτελεί ολοκληρομένη συνεισφορά καθώς είχαμε δώσει ιδιαίτερη έμφαση στο γεγονός οτι έχουμε το minimal-ionio theme σε ξεχωριστό αποθετήριο μαζί με τα includes του και τα συναφή. Αν ο λόγος που συνέβη αυτό βέβαια είναι για να διευκολυνθέι το minimal-ionio upgrade τότε δεκτό αλλά θα πρέπει να ξαναπροσαρμωστεί αμέσως. Ωστόσο δεν θα ήταν πιο σωστό να ήταν η πιο πρόσφατη ανακοίνωση και όχι κάποια τυχαία; Ξανά, συγχωρέστε με για το αργοπορημένο σχόλιο αλλά ήθελα να μοιραστώ τις σκέψεις μου και του @provopoulos. |
For clarity: |
Σε αυτά που είπε ο Δημήτρης θα ήθελα να προσθέσω μια πιθανή μελλοντική βελτίωση. Να υπάρχει κάποιο όριο, πιθανόν, εμφάνισης λέξεων ώστε να μην υπάρχει ασυμφωνία ευθυγράμμισης των στηλών, π.χ. https://sharp-borg-3fc7ea.netlify.com/ |
@JimDragon μπράβο σου για την αναλυτική περιγραφή, η οποία όμως για να είναι χρήσιμη στο έργο θα πρέπει να σπάσει σε επιμέρους τμήματα τα οποία θα γίνουν νέα θέματα σε συντονισμό με τον @provopoulos που φαίνεται να έχει βρει λύση για κάποια και με ανάθεση σε συντελεστές που θα μπορούσαν να βοηθήσουν προφανώς δεν έχει νόημα να τα συζητάμε εδώ, γιατί είναι σαν "να κλαίμε πάνω από το χυμένο γάλα", ας δώσουμε λύση καλύτερα! |
Include files are better off at the external theme repository.
Σχετικό Issue
closes #78
Προτεινόμενες Αλλαγές
-- Αλλαγή 1
-Δημιουργία φακέλου _includes και dynamic.html για την υλοποίηση της δυναμικής αλλαγής του περιεχομένου στην αρχική σελίδα
-- Αλλαγή 2
-Επεξεργασία του αρχείου index.html για την χρήση του dynamic.html για την εμφάνιση δυναμικού περιεχομένου
--Τελικό Αποτέλεσμα
Πλέον στην αρχική σελίδα εμφανίζονται με τυχαιότητα μία από τις ανακοινώσεις, ένα από τα projects και ένα από τα εργαστήρια. Το καθένα περιλαμβάνει ένα τίτλο και ένα μικρό περιλιπτικό κείμενο. Κλικάροντας στον τίτλο ο χρήστης οδηγείται στην σελίδα του αντίστοιχου περιεχομένου (ανακοίνωση ΧΧ, project ΧΧ ...) και κλικάροντας το "Learn More" κάτω από κάθε κατηγορία οδηγεί αντίστοιχα στην σελίδα με τα posts, στην σελίδα με τα projects ή στην σελίδα με τα εργαστήρια.
Ενδεικτικό link: https://sharp-borg-3fc7ea.netlify.com/
Υπενθυμίσεις